Conversation
…ttendanceType 을 enum 으로 구현하여 잘못된 입력 값 방지.
- 동사로 시작, get, create, update, delete 등으로 사용
- 코드 가독성을 높이기 위해 날짜를 year, month 변수로 분리 - ScheduleCategory 가 잘못된 값이 들어와도 값이 조회되도록 변경
- 조회수 부분 추가 - 정기운동, 대회, 친선 매치, 기타 부분만 완료된 것(실제 매치 부분은 리다이렉트 등 여러 방식 고민 중) - 모임에 소속되지 않은 사람이 조회하면 예외 발생하는 부분 테스트 코드 추가
- 일정 중 친선 매치/대회는 상대 팀 정보도 입력 받아야 하므로 save, update 메서드에 상대 팀 정보 받는 것 추가 - 외부 상대 팀 정보를 저장힉 위해 Schedule 엔티티에 상대 팀 정보 컬럼으로 추가(@embeddable 사용. 코드 재사용성을 위해 클래스 분리) - 일정 세부 조회 시 일정 종류에 따라 응답 값 다르도록 구현(아직 매치 부분은 구현 못함)
… 없어도, 있어도 조회도록 수정
- 있을 경우, 사용자가 수정 요청한 데이터만 변경되도록 setter 사용 - 없을 경우, 새로운 상대 팀 정보 객체 생성하여 할당 - 없을 경우, setter 사용되면 NullPointerException 발생하여 이렇게 로직 변경
Collaborator
|
타겟 브랜치 dev로 바꿔주셔야 할꺼같아요..! |
hojooo
approved these changes
May 31, 2025
Collaborator
hojooo
left a comment
hojooo
left a comment
There was a problem hiding this comment.
수고하셨어요! 항상 꼼꼼하고 잘 해주셔서 코드 보면서 많이 배우고 있습니다!!
y3binchoi
approved these changes
Jun 1, 2025
Collaborator
y3binchoi
left a comment
y3binchoi
left a comment
There was a problem hiding this comment.
수고하셨습니다!! ScheduleService 책임이 과하다는 생각이 들었는데 역시나 ScheduleCommandService, ScheduleQueryService로 잘 분산해주셔서 가독성과 성능 모두 좋아진 것 같습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
♻️ schedule 도메인 리팩터링 및 새로운 기능 추가
📌 변경 사항 (What’s changed?)
Schedule > Comment,Schedule > Vote이렇게 도메인 안에 속한 도메인 구조로 변경⚙️ 변경 이유 (Why?)
✅ 테스트 방법 (How to test?)
🤔 기타 참고 사항
💬리뷰 요구사항